Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added FieldHandler for field type ezlandingpage #153

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

christianxrow
Copy link

For ezsystems/landing-page-fieldtype-bundle used in eZ 6 / eZ Studio

@gggeek
Copy link
Member

gggeek commented Feb 10, 2018

I would have at a bare minimum make sure this is not tested by Travis...

Any idea of how we could possibly test against ezstudio ?

@xrow
Copy link
Contributor

xrow commented Feb 12, 2018

Any idea of how we could possibly test against ezstudio ?

Someone like @andrerom should tell us how this should be done for non public code. Andre, is there concept for it?

@christianxrow
Copy link
Author

@gggeek Unfortunately I don't know yet how to properly skip the test. Do you have any suggestions for me?

@andrerom
Copy link

andrerom commented Feb 12, 2018

We don't have a way for public non eZ code to test against non public eZ code.

For now you can detect if class is there and skip if not, in the future we might be able to find a way for this to be tests one way or another.

@xrow
Copy link
Contributor

xrow commented Feb 12, 2018

So we shall skip/not test for now.

@gggeek gggeek mentioned this pull request Aug 2, 2019
@gggeek gggeek changed the base branch from master to main October 13, 2022 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants